Skip to content

docs: modernize code examples#5687

Open
Copilot wants to merge 5 commits intomainfrom
copilot/review-docs-for-correctness
Open

docs: modernize code examples#5687
Copilot wants to merge 5 commits intomainfrom
copilot/review-docs-for-correctness

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 6, 2026

Description written manually by @mark-wiemer:

PR Checklist

Overview

Various minor improvements to make our code samples more modern :)

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Feb 6, 2026

CLA Not Signed

Copilot AI and others added 4 commits February 6, 2026 15:52
Co-authored-by: mark-wiemer <7833360+mark-wiemer@users.noreply.github.com>
Co-authored-by: mark-wiemer <7833360+mark-wiemer@users.noreply.github.com>
…rsions

Co-authored-by: mark-wiemer <7833360+mark-wiemer@users.noreply.github.com>
Co-authored-by: mark-wiemer <7833360+mark-wiemer@users.noreply.github.com>
Copilot AI changed the title [WIP] Review documentation for correctness and code snippets Modernize documentation code examples to ES modules and current JavaScript standards Feb 6, 2026
Copilot AI requested a review from mark-wiemer February 6, 2026 16:00
Copy link
Copy Markdown
Member

@mark-wiemer mark-wiemer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changes looking good so far :)

Copilot AI requested a review from mark-wiemer February 7, 2026 20:22
@mark-wiemer mark-wiemer changed the title Modernize documentation code examples to ES modules and current JavaScript standards docs: modernize code examples Feb 14, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.40%. Comparing base (bbd0ace) to head (2dd909d).
⚠️ Report is 64 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5687   +/-   ##
=======================================
  Coverage   88.40%   88.40%           
=======================================
  Files          66       66           
  Lines        4794     4794           
  Branches      978      978           
=======================================
  Hits         4238     4238           
  Misses        556      556           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@mark-wiemer mark-wiemer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will merge in 7 days unless someone stops me :)

@mark-wiemer mark-wiemer marked this pull request as ready for review February 14, 2026 07:57
Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those merges look scary 😄 but yes please!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants